Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates in response to source-build-internal team renamed to source-build #15148

Merged
merged 3 commits into from
Oct 10, 2024

Conversation

MichaelSimons
Copy link
Member

Related to dotnet/source-build#4645.

The proposed header changes made to eng/DotNetBuild.props and eng/SourceBuildPrebuiltBaseline.xml are what I plan to update all product repo's to.

@MichaelSimons MichaelSimons requested review from mmitche, a team, ellahathaway and mthalman October 8, 2024 19:59
@@ -6,31 +6,30 @@ This document provides guidelines on the types of source code that are permissib

## Binaries

### Policy
### Binary Policy
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There were a number of markdown linter violations in this file I decided to address since touching.

Copy link
Member Author

@MichaelSimons MichaelSimons Oct 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The file rename is not strictly required but I thought it was a good opportunity to cleanup.

ellahathaway
ellahathaway previously approved these changes Oct 8, 2024
mthalman
mthalman previously approved these changes Oct 8, 2024
@mmitche
Copy link
Member

mmitche commented Oct 9, 2024

@MichaelSimons I see a couple of remaining refs in .github/CODEOWNERS as well as src/Microsoft.DotNet.SourceBuild/tasks/src/UsageReport/ValidateUsageAgainstBaseline.cs

@MichaelSimons MichaelSimons dismissed stale reviews from mthalman and ellahathaway via b959a50 October 9, 2024 15:27
@MichaelSimons
Copy link
Member Author

@MichaelSimons I see a couple of remaining refs in .github/CODEOWNERS as well as src/Microsoft.DotNet.SourceBuild/tasks/src/UsageReport/ValidateUsageAgainstBaseline.cs

Fixed in b959a50.

@MichaelSimons MichaelSimons merged commit 47bbcb1 into dotnet:main Oct 10, 2024
11 checks passed
@MichaelSimons MichaelSimons deleted the sb-rename branch October 10, 2024 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants